Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ci: enable tests with go 1.19.x #82

Merged
merged 2 commits into from
Aug 30, 2022

Conversation

rst0git
Copy link
Member

@rst0git rst0git commented Aug 10, 2022

The verify / lint CI workflow running golangci-lint currently fails with:

phaul/api.go:30: File is not `gofmt`-ed with `-s` (gofmt)
//Methods:
phaul/server.go:37: File is not `gofmt`-ed with `-s` (gofmt)
//

This problem has been reported in golangci/golangci-lint-action#535

The problem occurs because, for some reason, golangci-lint is using gofmt from go 1.19. This pull request formats the code using gofmt and enables the CI tests to run with go 1.19.

This commit contains auto-generated changes with the following command:

    gofmt -w -s ./*/*.go

Signed-off-by: Radostin Stoyanov <rstoyanov@fedoraproject.org>
@codecov-commenter
Copy link

codecov-commenter commented Aug 10, 2022

Codecov Report

Merging #82 (7021e23) into master (9eea1cb) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #82   +/-   ##
=======================================
  Coverage   51.38%   51.38%           
=======================================
  Files           2        2           
  Lines         288      288           
=======================================
  Hits          148      148           
  Misses        105      105           
  Partials       35       35           

Help us with your feedback. Take ten seconds to tell us how you rate us.

The verify / lint CI workflow running golangci-lint currently fails
with:

	phaul/api.go:30: File is not `gofmt`-ed with `-s` (gofmt) //Methods:
	phaul/server.go:37: File is not `gofmt`-ed with `-s` (gofmt) //

The problem occurs because, for some reason, golangci-lint is using
gofmt from go 1.19. In a previous patch the code has been formatted with
go 1.19. This patch enables the CI tests to run with go 1.19.X.

Signed-off-by: Radostin Stoyanov <rstoyanov@fedoraproject.org>
@adrianreber adrianreber merged commit 634f2a5 into checkpoint-restore:master Aug 30, 2022
@rst0git rst0git deleted the ci-go-1.19 branch August 30, 2022 10:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants